Skip to content

Align workflow config spec with schema behavior#42

Merged
frantic-openai merged 1 commit intomainfrom
docs/spec-align-state-list-and-normalization
Mar 10, 2026
Merged

Align workflow config spec with schema behavior#42
frantic-openai merged 1 commit intomainfrom
docs/spec-align-state-list-and-normalization

Conversation

@frantic-openai
Copy link
Collaborator

Context

The schema and tests now treat tracker state lists as YAML lists and only lowercase
state-limit keys, but SPEC.md still documented older and stricter behavior.

TL;DR

Align SPEC.md with the current workflow config and state normalization behavior.

Summary

  • document tracker active_states and terminal_states as YAML string lists
  • update the cheat sheet defaults to use list examples instead of CSV strings
  • remove trim-based state normalization language from the spec

Alternatives

  • restore support for comma-separated tracker state strings in the schema
  • keep trim-based normalization in the spec and change the implementation

Test Plan

  • make -C elixir all
  • Docs-only diff reviewed against current schema and tests

Summary:
- document tracker active and terminal states as YAML string lists
- remove trim-based state normalization language from the spec
- update the config cheat sheet defaults to match list-based examples

Rationale:
- the schema now accepts tracker state lists rather than comma-separated
  strings, so the spec should not promise the older form
- the state normalization contract should match the less-defensive behavior
  the user wants documented

Tests:
- make -C elixir all

Co-authored-by: Codex <codex@openai.com>
@frantic-openai
Copy link
Collaborator Author

[codex] Changes in this PR:

  • Align tracker state list docs with the current schema
  • Update config cheat sheet defaults to list syntax
  • Remove trim-based state normalization language from SPEC.md
    Commits: 2f59b6a
    Tests: make -C elixir all

@frantic-openai frantic-openai merged commit 8afebb3 into main Mar 10, 2026
2 checks passed
@frantic-openai frantic-openai deleted the docs/spec-align-state-list-and-normalization branch March 10, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant